Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

website/integrations: added hoarder integration #12161

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

shelldandy
Copy link

@shelldandy shelldandy commented Nov 22, 2024

Details

Added Hoarder Integration to docs.

@shelldandy shelldandy requested a review from a team as a code owner November 22, 2024 17:30
Copy link

netlify bot commented Nov 22, 2024

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 258ac32
🔍 Latest deploy log https://app.netlify.com/sites/authentik-docs/deploys/6743f3cb3ef8960008ec77c8
😎 Deploy Preview https://deploy-preview-12161--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Nov 22, 2024

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit 258ac32
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/6743f3cba05a980008c3fb26

Copy link
Contributor

@4d62 4d62 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few comments. Let me know if you have any questions.

website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/sidebarsIntegrations.js Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
@shelldandy shelldandy requested a review from 4d62 November 24, 2024 22:37
@shelldandy
Copy link
Author

@4d62 just addressed the comments and commited the suggestions, lmk if more tweaks are needed

Copy link
Contributor

@4d62 4d62 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, thanks for your response & corrections. It looks good, but I noticed a few more things that can be changed to make it perfect.

cc @tanberry for final review

website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
website/integrations/services/hoarder/index.md Outdated Show resolved Hide resolved
shelldandy and others added 5 commits November 24, 2024 21:48
Co-authored-by: 4d62 <[email protected]>
Signed-off-by: Miguel Palau <[email protected]>
Co-authored-by: 4d62 <[email protected]>
Signed-off-by: Miguel Palau <[email protected]>
Co-authored-by: 4d62 <[email protected]>
Signed-off-by: Miguel Palau <[email protected]>
Co-authored-by: 4d62 <[email protected]>
Signed-off-by: Miguel Palau <[email protected]>
Co-authored-by: 4d62 <[email protected]>
Signed-off-by: Miguel Palau <[email protected]>
@shelldandy
Copy link
Author

@4d62 comments addressed lmk if more changes are needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants